home *** CD-ROM | disk | FTP | other *** search
- Subject: FW_CCommand::DoIt
- Sent: 6/2/96 1:45 AM
- Received: 6/3/96 7:58 AM
- From: Hutchings Software Development, hsd@earthlink.net
- Reply-To: ODF Interest, ODF-Interest@CILabs.ORG
- To: OpenDoc Development Framework Discussion List, ODF-Interest@CILabs.
-
- I noticed that FW_CCommand::DoIt is pure virtual. This caused a subtle
- bug for me this evening because I didn't initially realize that my
- override had
- to call "SaveUndoState" to make it get called!
-
- So, I eventually lifted the following out of
- FW_CClipboardCommand::DoIt.
-
- if (GetCanUndo(ev))
- this->SaveUndoState(ev); // save current state, for later Undo
-
- Would it be possible (for future release) to either move this to
- FW_CCommand::Execute, or put it in FW_CCommand:DoIt? It took me a good
- 15 minutes with MW Debug to figure out that I was missing those from
- my DoIt override. If the undo/redo methods are defined in FW_CCommand,
- why not wire them up too?
-
-
-
- Brad
-
-
-
-
- Richard B. "Brad" Hutchings
- General Partner,
- Hutchings Software Development
-
- hsd@earthlink.net
-
-
-